Skip to content

Cache result of validation #1730

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
Draft

Conversation

shmax
Copy link
Contributor

@shmax shmax commented Jul 11, 2025

Heyo, I remember there was some discussion about possibly disabling document validation checks a while back. I finally got around to looking into it and realized to my horror that it was indeed gobbling up a lot of resources in my app.

I don't know if you folks ever really agreed on a plan, but I thought I'd float this simple caching solution that leaves it up to the user.

Let me know what you think. If you like the general direction I can do a little polish and write better tests.

@shmax shmax changed the title cache result of validation Cache result of validation Jul 11, 2025
@shmax shmax force-pushed the validation-cache branch from 2d38364 to a2c38f4 Compare July 11, 2025 03:55
@shmax shmax force-pushed the validation-cache branch from 9ffbbd7 to 6782bad Compare July 11, 2025 04:07
@shmax shmax force-pushed the validation-cache branch from e3bc588 to 6317931 Compare July 12, 2025 00:05
@shmax shmax force-pushed the validation-cache branch from 1a59209 to 73c9774 Compare July 12, 2025 00:40
@shmax shmax force-pushed the validation-cache branch from 5e59b9c to 6d7ace0 Compare July 12, 2025 00:41
@shmax shmax force-pushed the validation-cache branch from 59f0ea0 to 708c85b Compare July 12, 2025 00:45
@shmax shmax force-pushed the validation-cache branch from a12854e to f35cb3c Compare July 12, 2025 01:53
@shmax shmax force-pushed the validation-cache branch from e6fa9a4 to d71e123 Compare July 12, 2025 04:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants